-
Notifications
You must be signed in to change notification settings - Fork 52
Optimized CI to reduce testing time #336
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… and unused variables
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request transitions the hypervisor from loading guest images from memory to loading them from the filesystem, and removes the CI workflow steps that previously downloaded and patched guest images. This optimization aims to reduce testing time by eliminating the download and patching operations.
- Changed all VM configurations to use filesystem-based image loading (
image_location = "fs") - Added filesystem and SD/MMC driver support to both
phytiumpiandroc-rk3568-pcboard configurations - Simplified CI workflow by removing guest image download and VM config patching steps
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
configs/board/phytiumpi.toml |
Added filesystem feature and SD/MMC and Phytium block drivers |
configs/board/roc-rk3568-pc.toml |
Added filesystem feature and SD/MMC and RK3568 clock drivers |
configs/vms/arceos-aarch64-e2000-smp1.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/arceos-aarch64-e2000-smp2.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/arceos-aarch64-rk3568-smp1.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/arceos-aarch64-rk3568-smp2.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/linux-aarch64-e2000-smp1.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/linux-aarch64-e2000-smp2.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/linux-aarch64-rk3568-smp1.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/linux-aarch64-rk3568-smp2.toml |
Changed to filesystem loading with standardized kernel path |
configs/vms/rtthread-aarch64-e2000-smp1.toml |
Changed to filesystem loading with standardized kernel path |
.github/workflows/test-board.yml |
Removed guest image download and config patching steps, cleaned up matrix entries |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This pull request updates the board and VM configurations to enable loading guest images from the file system instead of memory, and adds new features and drivers for both the
phytiumpiandroc-rk3568-pcboards. It also simplifies the CI workflow by removing image download and patch steps.Board and Feature Updates:
fs) and SD/MMC driver support to bothphytiumpiandroc-rk3568-pcboard configurations, along with board-specific block or clock drivers. [1] [2]VM Configuration Changes:
phytiumpiandroc-rk3568-pcto load kernel images from the file system (image_location = "fs") and updatedkernel_pathvalues to standardized guest image paths. [1] [2] [3] [4] [5] [6] [7] [8] [9]CI Workflow Simplification:
vmimage_namefields and related comments from the test matrix in.github/workflows/test-board.yml.